ComboBox: Remove pointless signal ID member fields
authorDaniel Boles <dboles@src.gnome.org>
Fri, 1 Sep 2017 10:52:12 +0000 (11:52 +0100)
committerDaniel Boles <dboles@src.gnome.org>
Fri, 1 Sep 2017 15:28:56 +0000 (16:28 +0100)
We don’t need to store these for disconnection, as we can just use
g_signal_handlers_disconnect_by_data().

gtk/gtkcombobox.c

index 474eb21da23dabda3a8a2e34928f82891ebbc5af..5e59260781622d54a12726174d30fa5fe953c094 100644 (file)
@@ -138,10 +138,6 @@ struct _GtkComboBoxPrivate
 
   GtkWidget *popup_widget;
 
-  gulong inserted_id;
-  gulong deleted_id;
-  gulong reordered_id;
-  gulong changed_id;
   guint popup_idle_id;
   guint scroll_timer;
   guint resize_idle_id;
@@ -1620,14 +1616,7 @@ gtk_combo_box_unset_model (GtkComboBox *combo_box)
 
   if (priv->model)
     {
-      g_signal_handler_disconnect (priv->model,
-                                   priv->inserted_id);
-      g_signal_handler_disconnect (priv->model,
-                                   priv->deleted_id);
-      g_signal_handler_disconnect (priv->model,
-                                   priv->reordered_id);
-      g_signal_handler_disconnect (priv->model,
-                                   priv->changed_id);
+      g_signal_handlers_disconnect_by_data (priv->model, combo_box);
 
       g_object_unref (priv->model);
       priv->model = NULL;
@@ -2469,22 +2458,18 @@ gtk_combo_box_set_model (GtkComboBox  *combo_box,
   priv->model = model;
   g_object_ref (priv->model);
 
-  priv->inserted_id =
-    g_signal_connect (priv->model, "row-inserted",
-                      G_CALLBACK (gtk_combo_box_model_row_inserted),
-                      combo_box);
-  priv->deleted_id =
-    g_signal_connect (priv->model, "row-deleted",
-                      G_CALLBACK (gtk_combo_box_model_row_deleted),
-                      combo_box);
-  priv->reordered_id =
-    g_signal_connect (priv->model, "rows-reordered",
-                      G_CALLBACK (gtk_combo_box_model_rows_reordered),
-                      combo_box);
-  priv->changed_id =
-    g_signal_connect (priv->model, "row-changed",
-                      G_CALLBACK (gtk_combo_box_model_row_changed),
-                      combo_box);
+  g_signal_connect (priv->model, "row-inserted",
+                    G_CALLBACK (gtk_combo_box_model_row_inserted),
+                    combo_box);
+  g_signal_connect (priv->model, "row-deleted",
+                    G_CALLBACK (gtk_combo_box_model_row_deleted),
+                    combo_box);
+  g_signal_connect (priv->model, "rows-reordered",
+                    G_CALLBACK (gtk_combo_box_model_rows_reordered),
+                    combo_box);
+  g_signal_connect (priv->model, "row-changed",
+                    G_CALLBACK (gtk_combo_box_model_row_changed),
+                    combo_box);
 
   _gtk_tree_menu_set_model (GTK_TREE_MENU (priv->popup_widget),
                             priv->model);